Skip to content

🐛 inference: fully-qualify curlimages/curl image for OL9 cri-o short-name enforcement - #17709

Merged
clubanderson merged 1 commit into
v5from
fix/inference-curl-fqdn-image
Nov 11, 2026
Merged

clubanderson merged 1 commit into
v5from
fix/inference-curl-fqdn-image

Conversation

@clubanderson

Copy link
Copy Markdown
Member

Why

After the hive-oke node pool moved to the OL 9.8 / OKE 1.34 image (#17642), hive-inference/vllm stuck in Init:ImageInspectError:

short name mode is enforcing, but image name curlimages/curl:8.13.0 returns ambiguous list

OL9 cri-o ships short-name-mode = "enforcing"; OL8 silently resolved the short name to Docker Hub.

What

  • src/deploy/inference/vllm-deployment.yaml: curlimages/curl:8.13.0@sha256:… → docker.io/curlimages/curl:8.13.0@sha256:… (same digest).
  • Live deploy on hive-oke already patched with kubectl set image; this keeps the checked-in manifest in sync.
  • This was the only unqualified image under src/deploy/.

Refs #17642.

…name enforcement

OL9 cri-o on OKE 1.34 node images enforces short-name resolution, so the
vLLM model-download init container failed with Init:ImageInspectError
("returns ambiguous list") after the hive-oke node cycling. Pin the
registry explicitly. Refs #17642.

Signed-off-by: Andrew Anderson <andy@clubanderson.com>
@kubestellar-prow kubestellar-prow Bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Nov 11, 2026
@clubanderson

Copy link
Copy Markdown
Member Author

/lgtm
/approve

@kubestellar-prow kubestellar-prow Bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Nov 11, 2026
@kubestellar-prow

Copy link
Copy Markdown
Contributor

@clubanderson: you cannot LGTM your own PR.

Details

In response to this:

/lgtm
/approve

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@kubestellar-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: clubanderson

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubestellar-prow kubestellar-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Open PR overlap check

Base branch: v5

Conflicts are checked with every PR merged onto the current tip of v5, so a PR that only needs a rebase is not reported as conflicting with its siblings.

No conflicting open PRs or same-file overlaps were found.

@clubanderson
clubanderson merged commit 0121e2e into v5 Nov 11, 2026
85 of 92 checks passed
@kubestellar-prow
kubestellar-prow Bot deleted the fix/inference-curl-fqdn-image branch November 11, 2026 17:13
@github-actions

Copy link
Copy Markdown
Contributor

Thank you for your contribution! Your PR has been merged.

We'd love to hear how your experience was: share feedback

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has signed the DCO. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant